Add explicit transfer annotations to gtk_icon_view_get_cursor
authorSimon Feltman <sfeltman@src.gnome.org>
Sun, 28 Jul 2013 20:16:16 +0000 (13:16 -0700)
committerSimon Feltman <sfeltman@src.gnome.org>
Mon, 29 Jul 2013 04:25:57 +0000 (21:25 -0700)
The cell out argument to gtk_icon_view_get_cursor is a pass-through
for gtk_cell_area_get_focus_cell which is transfer none. Without
this explicit annotation, transfer full is defaulted and introspection
bindings will assume ownership of the GtkIconViews reference to the
cell, causing crashes. Additionally add explicit transfer full to
the path parameter because it is expected the caller will free the
returned memory.

https://bugzilla.gnome.org/show_bug.cgi?id=704700

gtk/gtkiconview.c

index 049ccce4caa23d0aeaea17c01301c05183b983f9..f7e9ad80fcdd4dc1bcb10166f5bd5266c9bcef83 100644 (file)
@@ -2209,9 +2209,10 @@ gtk_icon_view_set_cursor (GtkIconView     *icon_view,
 /**
  * gtk_icon_view_get_cursor:
  * @icon_view: A #GtkIconView
- * @path: (out) (allow-none): Return location for the current cursor path,
- *        or %NULL
- * @cell: (out) (allow-none): Return location the current focus cell, or %NULL
+ * @path: (out) (allow-none) (transfer full): Return location for the current
+ *        cursor path, or %NULL
+ * @cell: (out) (allow-none) (transfer none): Return location the current
+ *        focus cell, or %NULL
  *
  * Fills in @path and @cell with the current cursor path and cell. 
  * If the cursor isn't currently set, then *@path will be %NULL.